Skip to content

Conversation

@pboling
Copy link
Member

@pboling pboling commented Jul 28, 2025

Two often conflicting goals resolved

  • deps_unlocked.yml

    • All runtime & dev dependencies, but does not have a gemfiles/*.gemfile.lock committed
    • Uses an Appraisal2 "deps_unlocked" gemfile, and the current MRI Ruby release
    • Know when new dependency releases will break local dev with unlocked dependencies
    • Broken workflow indicates that new releases of dependencies may not work
  • deps_locked.yml

    • All runtime & dev dependencies, and has a Gemfile.lock committed
    • Uses the project's main Gemfile, and the current MRI Ruby release
    • Matches what contributors and maintainers use locally for development
    • Broken workflow indicates that a new contributor will have a bad time

@kou - FYI, This is the pattern I would have used in ruby/rexml#279, had I completed it.

@pboling pboling self-assigned this Jul 28, 2025
@coveralls
Copy link

coveralls commented Jul 28, 2025

Pull Request Test Coverage Report for Build 16581131935

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 89.912%

Totals Coverage Status
Change from base Build 16577499423: 0.0%
Covered Lines: 348
Relevant Lines: 378

💛 - Coveralls

@pboling pboling changed the title 👷 Using released appraisal2 allows for simpler config 👷 Lock/Unlock Deps Pattern Jul 28, 2025
Two often conflicting goals resolved

- deps_unlocked.yml
  - All runtime & dev dependencies, but does not have a `gemfiles/*.gemfile.lock` committed
  - Uses an Appraisal2 "deps_unlocked" gemfile, and the current MRI Ruby release
  - Know when new dependency releases will break local dev with unlocked dependencies
  - Broken workflow indicates that new releases of dependencies may not work

- deps_locked.yml
  - All runtime & dev dependencies, and has a `Gemfile.lock` committed
  - Uses the project's main Gemfile, and the current MRI Ruby release
  - Matches what contributors and maintainers use locally for development
  - Broken workflow indicates that a new contributor will have a bad time
@github-actions
Copy link

Code Coverage

Package Line Rate Branch Rate Health
omniauth-identity 92% 79%
Summary 92% (348 / 378) 79% (62 / 78)

Minimum allowed line rate is 90%

@pboling pboling merged commit 259197b into main Jul 28, 2025
68 checks passed
@pboling pboling deleted the simplify-ci branch July 28, 2025 21:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants